Ff 1471 #40
Ff 1471 #40
Conversation
eversonbueno
requested review from
renatops1991,
PedroFellipe,
rorteg,
ran-j and
felipemaksy
as code owners
|
Vai ser necessário deixar esse PR bem preenchido, de acordo com o template. |
|
Acredito que seja interessante adicionar alguns testes e dataprovider, em caso de dúvida me procure inbox que eu contextualizo melhor. |
|
Revisar a estrutura para se adequar aos guidelines da linguagem: https://github.com/golang-standards/project-layout. |
.env.example
Outdated
| # Configs LocalStack | ||
| LOCALSTACK_HOST=http://0.0.0.0:4566 | ||
| LOCALSTACK_SQS_HOST=http://0.0.0.0:4566/000000000000/test-queue |
There was a problem hiding this comment.
Precisamos reduzir a quantidade de variaveis em relação ao localstack, por exemplo, a variável da região é uma variável global que atende a outras integrações, então não compensa existir LOCALSTACK_SQS_REGION.
Podemos ter algo nesse sentido:
LOG_LEVEL=debug
REGION_NAME=us-east-1
SQS_ENDPOINT=http://localstack:4566
SQS_LOCALSTACK=http://localstack:4566
LOCALSTACK_ENDPOINT=http://localstack:4566
APP_QUEUE=http://localstack:4566/000000000000/test-queue
There was a problem hiding this comment.
Estuda a possibilidade de centralizar o arquivo de variáveis de ambiente na pasta /env/. assim vamos atender uma guideline da MM.
Aqui segue um exemplo: https://github.com/madeiramadeirabr/template-serverless-lambda-python/tree/production/examples/lambda_sqs/env
| - 8080:8080 | ||
|
|
||
| localstack: |
There was a problem hiding this comment.
Precisa incluir umas instruções a mais:
Veja esse arquivo: https://github.com/madeiramadeirabr/template-serverless-lambda-python/blob/production/examples/lambda_sqs/docker-compose.yml#L41
There was a problem hiding this comment.
pkg/cloud/aws/sqs.go
Outdated
| }, nil | ||
| } | ||
|
|
||
| func (s SQS) Delete(ctx context.Context, queueURL, rcvHandle string) error { |
There was a problem hiding this comment.
Acredito que podemos ter operações para a fila e para as mensagens, exemplo: Delete e DeleteQueue, sei que já está praticamente neste formato, talvez acrescentar a função/método faltante. Operações de deleção lembre de sempre retornar um bool para indicar o sucesso da operação.
pkg/cloud/client.go
Outdated
| // Long polls given amount of messages from a queue. | ||
| Receive(ctx context.Context, queueURL string) (*Message, error) | ||
| // Deletes a message from a queue. | ||
| Delete(ctx context.Context, queueURL, rcvHandle string) error |
There was a problem hiding this comment.
Talvez acrescentar alguns sufixos como Message e Queue, exemplo:
SendMessage(), ReceiveMessage(), DeleteQueue(), DeleteMessage(), CreateQueue()
pkg/cloud/model.go
Outdated
| @@ -0,0 +1,20 @@ | |||
| package cloud | |||
|
|
|||
| type SendRequest struct { | |||
There was a problem hiding this comment.
Ao meu ver, como fazem parte do contexto de SQS, me parece mais sensato estar no mesmo pacote "aws"
| @@ -0,0 +1,30 @@ | |||
| #!/bin/bash | |||
| # ************************** | |||
There was a problem hiding this comment.
Vamos precisar aplicar umas atualizações nesse carinha, me chame inbox para te ajudar nesse processo.
| @@ -0,0 +1,12 @@ | |||
| #!/bin/bash | |||
| # ************************** | |||
There was a problem hiding this comment.
Vamos precisar atualizar também.
| #!/bin/bash | ||
| # ************************** | ||
| # Localstack SQS Receive Messages Tool | ||
| # Version: 1.0.0 |
There was a problem hiding this comment.
Vamos precisar atualizar também.
| @@ -0,0 +1,73 @@ | |||
| #!/bin/bash | |||
There was a problem hiding this comment.
Vamos precisar atualizar também.
| @@ -1,4 +1,4 @@ | |||
| #!/bin/sh | |||
| #!/bin/sh | |||
There was a problem hiding this comment.
Precisamos identificar o uso desse script, quem sabe vamos necessitar mover para a pasta ./scripts/tools
|
Kudos, SonarCloud Quality Gate passed! |
[COREAPI-XXXX]
Proposed changes
Describe your changes briefly and objectively.
Checklist